Skip to content

feat(skill-cleaner): add exclusive root scans - #19

Merged
steipete merged 1 commit into
mainfrom
fix/skill-cleaner-root-only
Jul 1, 2026
Merged

feat(skill-cleaner): add exclusive root scans#19
steipete merged 1 commit into
mainfrom
fix/skill-cleaner-root-only

Conversation

@steipete

@steipete steipete commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • add backward-compatible --root-only mode for isolated skill inventories
  • require at least one explicit --root and skip live/default Codex roots in exclusive mode
  • preserve additive --root behavior, add regression coverage, and document the new workflow

Closes #17

Proof

  • node --test --experimental-strip-types skills/skill-cleaner/scripts/skill-cleaner.test.ts
  • scripts/validate-skills
  • YAML frontmatter parse for skills/skill-cleaner/SKILL.md
  • exact CLI proof: exclusive scan returned only skills/skill-cleaner; missing --root exited 2
  • CI-equivalent shell syntax, ClawSweeper helper, npm install/audit, and video-transcript smoke checks
  • autoreview: clean, no accepted/actionable findings
  • Public Model Identifier Gate: PASS; no model identifiers added

@steipete
steipete merged commit 0e8ca00 into main Jul 1, 2026
2 checks passed
@steipete

steipete commented Jul 1, 2026

Copy link
Copy Markdown
Owner Author

Landed as 0e8ca002.

Proof:

  • Unit: node --test --experimental-strip-types skills/skill-cleaner/scripts/skill-cleaner.test.ts (6/6 passed).
  • Live CLI: --root skills/skill-cleaner --root-only --no-logs --json returned only skill-cleaner from the supplied root; --root-only without --root exited 2 with the documented error.
  • Validation: YAML frontmatter parsed; scripts/validate-skills validated 58 skills; diff check, shell syntax, ClawSweeper helper, npm audit/install, and video-transcript smoke passed.
  • Review: Codex autoreview reported no accepted/actionable findings.
  • CI: smoke and GitGuardian passed on exact head e9506809cde27a13c22d39d0bd07fa361caafb62.
  • Public Model Identifier Gate: PASS; no model identifiers added.

Caveats: none.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: --exclude-codex flag or exclusive --root mode for OpenCode-first users

1 participant